Skip to content

GH-50588: [Ruby] Add ArrowFormat::Array#[] - #50589

Merged
kou merged 1 commit into
apache:mainfrom
kou:ruby-array-aref
Jul 22, 2026
Merged

GH-50588: [Ruby] Add ArrowFormat::Array#[]#50589
kou merged 1 commit into
apache:mainfrom
kou:ruby-array-aref

Conversation

@kou

@kou kou commented Jul 22, 2026

Copy link
Copy Markdown
Member

Rationale for this change

It's a convenient API to get a value at the given index.

What changes are included in this PR?

Add ArrowFormat::Array#[].

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

Copilot AI review requested due to automatic review settings July 22, 2026 02:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50588 has been automatically assigned in GitHub to PR creator.

Copilot AI review requested due to automatic review settings July 22, 2026 03:26
@kou
kou force-pushed the ruby-array-aref branch from b6a4bd7 to 0673e25 Compare July 22, 2026 03:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kou

kou commented Jul 22, 2026

Copy link
Copy Markdown
Member Author

+1

@kou
kou merged commit 154962c into apache:main Jul 22, 2026
33 checks passed
@kou
kou deleted the ruby-array-aref branch July 22, 2026 04:42
@kou kou removed the awaiting committer review Awaiting committer review label Jul 22, 2026
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 154962c.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 13 possible false positives for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants